From 56bcb1933f6de613e5d8689e23420d47b65425c3 Mon Sep 17 00:00:00 2001 From: Federico Mena Quintero Date: Tue, 5 Mar 2013 13:59:04 -0600 Subject: [PATCH] filechooser: Rename _gtk_file_is_path_not_local() to _gtk_file_has_native_path() Negatives in names of boolean functions are confusing. Signed-off-by: Federico Mena Quintero --- gtk/gtkfilechooserdefault.c | 10 +++++----- gtk/gtkfilechooserentry.c | 2 +- gtk/gtkfilesystem.c | 11 +++++------ gtk/gtkfilesystem.h | 2 +- 4 files changed, 12 insertions(+), 13 deletions(-) diff --git a/gtk/gtkfilechooserdefault.c b/gtk/gtkfilechooserdefault.c index 8f0b278903..55788e3669 100644 --- a/gtk/gtkfilechooserdefault.c +++ b/gtk/gtkfilechooserdefault.c @@ -1854,7 +1854,7 @@ shortcuts_append_bookmarks (GtkFileChooserDefault *impl, file = bookmarks->data; - if (impl->local_only && _gtk_file_is_path_not_local (file)) + if (impl->local_only && !_gtk_file_has_native_path (file)) continue; if (shortcut_find_position (impl, file) != -1) @@ -1976,16 +1976,16 @@ shortcuts_add_volumes (GtkFileChooserDefault *impl) if (_gtk_file_system_volume_is_mounted (volume)) { GFile *base_file; - gboolean base_is_not_local = FALSE; + gboolean base_has_native_path = FALSE; base_file = _gtk_file_system_volume_get_root (volume); if (base_file != NULL) { - base_is_not_local = _gtk_file_is_path_not_local (base_file); + base_has_native_path = _gtk_file_has_native_path (base_file); g_object_unref (base_file); } - if (base_is_not_local) + if (!base_has_native_path) continue; } } @@ -7338,7 +7338,7 @@ gtk_file_chooser_default_update_current_folder (GtkFileChooser *chooser, operation_mode_set (impl, OPERATION_MODE_BROWSE); - if (impl->local_only && _gtk_file_is_path_not_local (file)) + if (impl->local_only && !_gtk_file_has_native_path (file)) { g_set_error_literal (error, GTK_FILE_CHOOSER_ERROR, diff --git a/gtk/gtkfilechooserentry.c b/gtk/gtkfilechooserentry.c index 603516a2cf..ce2b6cc9e7 100644 --- a/gtk/gtkfilechooserentry.c +++ b/gtk/gtkfilechooserentry.c @@ -558,7 +558,7 @@ set_completion_folder (GtkFileChooserEntry *chooser_entry, { if (folder_file && chooser_entry->local_only - && _gtk_file_is_path_not_local (folder_file)) + && !_gtk_file_has_native_path (folder_file)) folder_file = NULL; if ((chooser_entry->current_folder_file diff --git a/gtk/gtkfilesystem.c b/gtk/gtkfilesystem.c index db09c9a654..2fdfa3df9d 100644 --- a/gtk/gtkfilesystem.c +++ b/gtk/gtkfilesystem.c @@ -1274,16 +1274,15 @@ _gtk_file_info_consider_as_directory (GFileInfo *info) } gboolean -_gtk_file_is_path_not_local (GFile *file) +_gtk_file_has_native_path (GFile *file) { char *local_file_path; - gboolean is_not_local; + gboolean has_native_path; - /* Don't use is_native(), as we want to support fuse paths if available */ + /* Don't use g_file_is_native(), as we want to support FUSE paths if available */ local_file_path = g_file_get_path (file); - is_not_local = (local_file_path == NULL); + has_native_path = (local_file_path != NULL); g_free (local_file_path); - return is_not_local; + return has_native_path; } - diff --git a/gtk/gtkfilesystem.h b/gtk/gtkfilesystem.h index 57b687615b..03782d6f7f 100644 --- a/gtk/gtkfilesystem.h +++ b/gtk/gtkfilesystem.h @@ -129,7 +129,7 @@ GdkPixbuf * _gtk_file_info_render_icon (GFileInfo *info, gboolean _gtk_file_info_consider_as_directory (GFileInfo *info); /* GFile helper functions */ -gboolean _gtk_file_is_path_not_local (GFile *file); +gboolean _gtk_file_has_native_path (GFile *file); G_END_DECLS -- 2.30.2